fix: failed to build instrument hooks on bare metal runner#55
Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses build failures on bare-metal runners by ensuring the Go runner can compile the cgo-based instrumentation hooks (fixes #53).
Changes:
- Add an early check that a C compiler is available before proceeding with overlay/hook setup.
- Force
CGO_ENABLED=1for thego testinvocation so the cgo-based overlay file isn’t silently excluded. - Add agent-facing documentation (
AGENTS.md) and updateCLAUDE.mdaccordingly.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| go-runner/src/runner/mod.rs | Adds C compiler preflight checks and forces CGO_ENABLED=1 for benchmark builds. |
| AGENTS.md | Documents project architecture, runner modes, and build/test workflows. |
| CLAUDE.md | Points Claude/agents to the agent documentation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Merging this PR will not alter performance
Comparing 
Footnotes
|
not-matthias
force-pushed
the
cod-2306-failed-to-build-instrument-hooks-on-bare-metal-runner
branch
from
6df525d to
9e139ac
Compare
not-matthias
deleted the
cod-2306-failed-to-build-instrument-hooks-on-bare-metal-runner
branch
Fixes #53